Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapt rust tests to test suite refactor #81

Merged
merged 3 commits into from
Dec 29, 2024

Conversation

marijncv
Copy link
Collaborator

Summary & Motivation

This aligns the Cli struct with the cli arguments passed in the test suite

How I Tested These Changes

cd libraries/pipes/implementations/rust
uv run pytest

Changelog

Ensure that an entry has been created in CHANGELOG.md outlining additions, deletions, and/or modifications.

See: keepachangelog.com

@danielgafni
Copy link
Contributor

Sorry @marijncv, could you please fix the conflict? I'll merge after that

@marijncv
Copy link
Collaborator Author

Sorry @marijncv, could you please fix the conflict? I'll merge after that

No worries, should be fixed now!

@cmpadden
Copy link
Collaborator

Sorry @marijncv, could you please fix the conflict? I'll merge after that

No worries, should be fixed now!

Really impressed by the work you two are doing. Going to merge this into refactor-pipes-tests now that the conflict has been resolved.

@cmpadden cmpadden merged commit 8f68d3e into refactor-pipes-tests Dec 29, 2024
3 checks passed
@cmpadden cmpadden deleted the refactor-pipes-tests-rust branch December 29, 2024 01:30
cmpadden pushed a commit that referenced this pull request Jan 9, 2025
* [pipes] refactor Pipes tests interface

* Adapt rust tests to test suite refactor (#81)

* [pipes] refactor Pipes tests interface

* adapt rust tests to test suite refactor

---------

Co-authored-by: danielgafni <[email protected]>

* add missing extras

* Java tests refactoring (#84)

* java tests refactoring

---------

Co-authored-by: Marijn Valk <[email protected]>
Co-authored-by: Andrey Grishchenko <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants